Skip to content

Conversation

@y-rabie
Copy link
Contributor

@y-rabie y-rabie commented Nov 19, 2025

Second and last PR split from #7307 (includes the changes of #7557, will rebase when it's merged)
E2e tests should be successful here.

@codecov
Copy link

codecov bot commented Nov 19, 2025

Codecov Report

❌ Patch coverage is 18.61702% with 153 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.25%. Comparing base (0fa26d7) to head (ae16e7d).
⚠️ Report is 13 commits behind head on main.

Files with missing lines Patch % Lines
internal/gatewayapi/runner/runner.go 0.00% 121 Missing ⚠️
internal/provider/kubernetes/status.go 36.17% 30 Missing ⚠️
internal/gatewayapi/extensionserverpolicy.go 90.00% 1 Missing and 1 partial ⚠️

❌ Your patch check has failed because the patch coverage (18.61%) is below the target coverage (60.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7558      +/-   ##
==========================================
- Coverage   72.43%   72.25%   -0.18%     
==========================================
  Files         232      232              
  Lines       34307    34392      +85     
==========================================
+ Hits        24849    24851       +2     
- Misses       7682     7761      +79     
- Partials     1776     1780       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@y-rabie y-rabie force-pushed the merge-status-across-controllers branch from 6ffa015 to 9769a3c Compare November 19, 2025 10:00
@y-rabie y-rabie changed the title fix: merge xRoute/xPolicy statuses across controllers with provider runner fix: merge xRoute/xPolicy statuses across controllers in provider runner Nov 19, 2025
@y-rabie y-rabie force-pushed the merge-status-across-controllers branch 2 times, most recently from 9abf08f to 02d6094 Compare December 5, 2025 01:35
@y-rabie
Copy link
Contributor Author

y-rabie commented Dec 5, 2025

/retest

@y-rabie y-rabie force-pushed the merge-status-across-controllers branch from 02d6094 to ae16e7d Compare December 5, 2025 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant